Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nordvpn: 8.17.1 and update URL #169148

Merged
merged 1 commit into from
Mar 18, 2024
Merged

Conversation

adithyasunil26
Copy link
Contributor

After making any changes to a cask, existing or new, verify:

@p-linnane
Copy link
Member

Can you explain the motivation for this change? Are upstream not offering versioned releases anymore?

@p-linnane p-linnane added the awaiting user reply Issue needs response from a user. label Mar 15, 2024
@adithyasunil26
Copy link
Contributor Author

The latest version seems to be unavailable through the versioned URL.

@krehel
Copy link
Member

krehel commented Mar 17, 2024

This is a rather unfavorable regression.... it would be great if we can uncover a versioned link. I did some poking but couldn't find one immediately. Will keep looking... but we may have to take this one as-is.

@bevanjkay
Copy link
Member

I can't find a versioned option ATM either, let's keep an eye on it and maybe in future the versioned url will return.

@bevanjkay bevanjkay enabled auto-merge March 18, 2024 22:06
@bevanjkay bevanjkay merged commit 5e82a38 into Homebrew:master Mar 18, 2024
11 checks passed
github-actions bot pushed a commit to MPLew-is/homebrew-vscodium that referenced this pull request Mar 19, 2024
…nordvpn-8.17.1

nordvpn: 8.17.1 and update URL
@adithyasunil26 adithyasunil26 deleted the nordvpn-8.17.1 branch March 19, 2024 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting user reply Issue needs response from a user.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants